-
Notifications
You must be signed in to change notification settings - Fork 2.8k
test(react19-tests-v9): run integration tests type-check against source #34864
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
test(react19-tests-v9): run integration tests type-check against source #34864
Conversation
📊 Bundle size reportUnchanged fixtures
|
Pull request demo site: URL |
bd93682
to
1c0cd52
Compare
@@ -52,7 +53,7 @@ export const useAccordion_unstable = <Value = AccordionItemValue>( | |||
}, | |||
root: slot.always( | |||
getIntrinsicElementProps('div', { | |||
...props, | |||
...rest, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
custom onToggle
callback conflicts with the native HTML one
e20be29
to
7494b8f
Compare
@@ -18,7 +18,7 @@ export type InfoButtonSlots = { | |||
/** | |||
* InfoButton Props | |||
*/ | |||
export type InfoButtonProps = Omit<ComponentProps<Partial<InfoButtonSlots>>, 'disabled'> & { | |||
export type InfoButtonProps = Omit<ComponentProps<Partial<InfoButtonSlots>>, 'disabled' | 'popover'> & { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The popover
prop clashes with the HTMLElement's popover
attrbiute
packages/react-components/react-jsx-runtime/src/jsx-runtime/index.d.ts__tmpl__
Outdated
Show resolved
Hide resolved
packages/react-components/react-motion/library/src/components/PresenceGroup.tsx
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
few things need to be reverted/changed otherwise good to go
packages/react-components/react-motion/library/src/factories/createPresenceComponent.ts
Outdated
Show resolved
Hide resolved
8b44316
to
f007f24
Compare
fbe712e
to
40cc9c2
Compare
…s-ignore for React.Component constructor
@@ -3,7 +3,7 @@ | |||
"target": "ES2019", | |||
"module": "esnext", | |||
"moduleResolution": "node", | |||
"lib": ["ES2019", "dom"], | |||
"lib": ["ES2019", "ES2020.Intl", "dom"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ES2020.Intl
is needed for time-picker-compat
https://github.com/microsoft/fluentui/blob/master/packages/react-components/react-timepicker-compat/library/tsconfig.lib.json#L5
Previous Behavior
React 19
type-check:integartion
task were run against components stories onlyNew Behavior
React 19
type-check:integartion
task is using both stories and packages source.Fixed type issues:
RefObject
no longer makes current nullable by default https://github.com/eps1lon/types-react-codemod?tab=readme-ov-file#refobject-defaults-react-19useRef
now always requires an initial value. Implicitundefined
is forbidden. https://github.com/eps1lon/types-react-codemod?tab=readme-ov-file#useref-required-initial-react-19Related Issue(s)
Blocked by chore: fix react 19 integration tests type issues for v9 #34722